Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Using string.match not regexp.test #57

Merged
merged 8 commits into from
Apr 11, 2016
Merged

Using string.match not regexp.test #57

merged 8 commits into from
Apr 11, 2016

Conversation

kfbishop
Copy link
Contributor

[My first PR, please forgive any failings]
In the demo, I sorted the cities array such that the RegExp.test() shows the failure, try searching on "Ma" will result in Madrid showing, Manchester not showing.

Ref Issue: #56

'Paris', 'Poznań', 'Prague', 'Riga', 'Rome', 'Rotterdam', 'Seville', 'Sheffield',
'Sofia', 'Stockholm', 'Stuttgart', 'The Hague', 'Turin', 'Valencia', 'Vienna',
'Vilnius', 'Warsaw', 'Wrocław', 'Zagreb', 'Zaragoza'];
private items:Array<string> = ['Amsterdam', 'Antwerp', 'Athens', 'Barcelona', 'Berlin', 'Birmingham', 'Bradford', 'Bremen', 'Brussels', 'Bucharest', 'Budapest', 'Cologne', 'Copenhagen', 'Dortmund', 'Dresden', 'Dublin', 'Düsseldorf', 'Essen', 'Frankfurt', 'Genoa', 'Glasgow', 'Gothenburg', 'Hamburg', 'Hannover', 'Helsinki', 'Kraków', 'Leeds', 'Leipzig', 'Lisbon', 'London', 'Madrid', 'Manchester', 'Marseille', 'Milan', 'Munich', 'Málaga', 'Naples', 'Palermo', 'Paris', 'Poznań', 'Prague', 'Riga', 'Rome', 'Rotterdam', 'Seville', 'Sheffield', 'Sofia', 'Stockholm', 'Stuttgart', 'The Hague', 'Turin', 'Valencia', 'Vienna', 'Vilnius', 'Warsaw', 'Wrocław', 'Zagreb', 'Zaragoza', 'Łódź'];

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

@kfbishop
Copy link
Contributor Author

There's still issues here with the rich options. The highlight pipe can break it, and also the value wont show HTML, so get raw markup once selected. Might want to hold off on this PR for now ... Thoughts and ideas welcomed though.

@valorkin valorkin self-assigned this Feb 22, 2016
@valorkin
Copy link
Member

@kfbishop first of all thanks for your time and efforts! :)
I will check it asap,
if you will squash and force push commits to your branch it will help a lot
with commit name
fix(select): invalid filtering due to usage of regexp.test. fixes #56
thanks in advance

@kfbishop
Copy link
Contributor Author

Did 2 squashes for #54 and #56. Sorry for the combined PR. Will use branches next time.

@valorkin valorkin merged commit 5c96c2d into valor-software:master Apr 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants